Skip to content

Light Freshness Edit: ASP.NET - migration & tutorials#37136

Open
lootle1 wants to merge 3 commits into
dotnet:mainfrom
lootle1:FR53
Open

Light Freshness Edit: ASP.NET - migration & tutorials#37136
lootle1 wants to merge 3 commits into
dotnet:mainfrom
lootle1:FR53

Conversation

@lootle1
Copy link
Copy Markdown
Contributor

@lootle1 lootle1 commented May 12, 2026 •
edited by github-actions Bot
Loading

This PR contains freshness updates to the following articles:

  • Migrate from ASP.NET Core in .NET 8 to ASP.NET Core in .NET 9
  • Get started with NSwag and ASP.NET Core
  • Part 7, add a new field to a Razor Page in ASP.NET Core
  • Tutorial: Call an ASP.NET Core web API with JavaScript

Minor grammar/style updates. Link updates.
For request details, see 568764

Internal previews

📄 File 🔗 Preview link
aspnetcore/migration/80-to-90.md Migrate from ASP.NET Core in .NET 8 to ASP.NET Core in .NET 9
aspnetcore/tutorials/getting-started-with-NSwag.md Get started with NSwag and ASP.NET Core
aspnetcore/tutorials/razor-pages/new-field.md [Visual Studio](https://review.learn.microsoft.com/en-us/aspnet/core/tutorials/razor-pages/new-field?branch=pr-en-us-37136)
aspnetcore/tutorials/web-api-javascript.md "Tutorial: Call an ASP.NET Core web API with JavaScript"

@lootle1 lootle1 marked this pull request as ready for review May 12, 2026 16:05
@lootle1
Copy link
Copy Markdown
Contributor Author

lootle1 commented May 12, 2026

@wadepickett this PR is ready for review. Note I recreated this PR from the previous one as there was a build error but persists here. It looks to be related to aspnetcore/mobile/native-mobile-backend.md, which isn't a part of my edits. Are you able to merge this or should I troubleshoot further to see what's happening?

@wadepickett
Copy link
Copy Markdown
Contributor

wadepickett commented May 13, 2026

Closing and reopening to force new build. I think the code snippet issue on main may have already been taken care of by another PR I submitted today,

@wadepickett wadepickett reopened this May 13, 2026
@wadepickett wadepickett self-requested a review May 13, 2026 01:03
wadepickett
wadepickett reviewed May 13, 2026
View reviewed changes
Comment thread aspnetcore/tutorials/getting-started-with-NSwag.md Outdated
wadepickett
wadepickett reviewed May 13, 2026
View reviewed changes
Comment thread aspnetcore/tutorials/getting-started-with-NSwag.md Outdated
wadepickett
wadepickett requested changes May 13, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lootle1, looks great! Just a couple items to address which I suggested inline. Thanks.

lootle1 and others added 2 commits May 13, 2026 09:39
@lootle1 @wadepickett
Update aspnetcore/tutorials/getting-started-with-NSwag.md
64fff41 
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
@lootle1 @wadepickett
Update aspnetcore/tutorials/getting-started-with-NSwag.md
0a42959 
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
@lootle1 lootle1 closed this May 13, 2026
@lootle1 lootle1 reopened this May 13, 2026
@lootle1
Copy link
Copy Markdown
Contributor Author

lootle1 commented May 13, 2026

@wadepickett thanks I've got those changes from you added. Still getting this error, now I believe the warnings are related to aspnetcore/host-and-deploy/health-checks/includes/health-checks6-7.md and aspnetcore/host-and-deploy/health-checks.md. These weren't a part of this PR so unsure why it's flagging but curious if you can merge despite that.

@MeeraDi MeeraDi mentioned this pull request May 14, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wadepickett wadepickett wadepickett requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lootle1 @wadepickett